Skip to content

[WIP] Fix search function layout on cluster page#654

Merged
LukasWallrich merged 3 commits intocopilot/make-cluster-page-searchablefrom
copilot/sub-pr-616
Feb 18, 2026
Merged

[WIP] Fix search function layout on cluster page#654
LukasWallrich merged 3 commits intocopilot/make-cluster-page-searchablefrom
copilot/sub-pr-616

Conversation

Copy link
Contributor

Copilot AI commented Feb 18, 2026

  • Fix close button visibility behind navbar by increasing z-index (toggle: 1034, panel: 1035)
  • Fix scroll behavior to show results at top of viewport with navbar offset

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits February 18, 2026 17:09
Co-authored-by: LukasWallrich <60155545+LukasWallrich@users.noreply.github.com>
@LukasWallrich LukasWallrich marked this pull request as ready for review February 18, 2026 17:10
@LukasWallrich LukasWallrich requested a review from a team as a code owner February 18, 2026 17:10
@LukasWallrich LukasWallrich merged commit 1d0a2a9 into copilot/make-cluster-page-searchable Feb 18, 2026
2 checks passed
@LukasWallrich LukasWallrich deleted the copilot/sub-pr-616 branch February 18, 2026 17:10
@github-actions
Copy link
Contributor

📝 Spell Check Results

Found 5 potential spelling issue(s) in this PR:

📄 content/glossary/english/open_material.md

Line Issue
16 Wit ==> With

📄 content/glossary/english/plan_s.md

Line Issue
4 commited ==> committed

📄 content/glossary/english/pro.md

Line Issue
4 publically ==> publicly

📄 content/glossary/english/research_protocol.md

Line Issue
4 publically ==> publicly

📄 content/glossary/english/type_i_error.md

Line Issue
4 occured ==> occurred

ℹ️ How to address these issues:

  1. Fix the typo: If it's a genuine typo, please correct it.
  2. Add to whitelist: If it's a valid word (e.g., a name, technical term), add it to .codespell-ignore.txt
  3. False positive: If this is a false positive, please report it in the PR comments.

🤖 This check was performed by codespell

@github-actions
Copy link
Contributor

👍 All image files/references (if any) are in webp format, in line with our policy.

Copilot AI requested a review from LukasWallrich February 18, 2026 17:21
Copilot stopped work on behalf of LukasWallrich due to an error February 18, 2026 17:22
richarddushime added a commit that referenced this pull request Mar 22, 2026
* Initial plan

* Add search functionality to clusters page for searching within tabs

Co-authored-by: LukasWallrich <60155545+LukasWallrich@users.noreply.github.com>

* Fix code review issues: improve regex handling and DOM manipulation

Co-authored-by: LukasWallrich <60155545+LukasWallrich@users.noreply.github.com>

* Redesign cluster search as collapsible sidebar (#650)

* Initial plan

* Redesign cluster search as collapsible left sidebar

Co-authored-by: LukasWallrich <60155545+LukasWallrich@users.noreply.github.com>

* Fix code review feedback: use alert-info for no results and prevent layout shift

Co-authored-by: LukasWallrich <60155545+LukasWallrich@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: LukasWallrich <60155545+LukasWallrich@users.noreply.github.com>

* [WIP] Fix search function layout on cluster page (#654)

* Initial plan

* Fix search panel visibility and scroll behavior

Co-authored-by: LukasWallrich <60155545+LukasWallrich@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: LukasWallrich <60155545+LukasWallrich@users.noreply.github.com>
Co-authored-by: Lukas Wallrich <lukas.wallrich@gmail.com>

* fix: improve clusters search panel UX

- Auto-hide panel when clicking search result
- Position panel below navbar to avoid hiding logo
- Reduce panel width to 70% on mobile devices
- Show only search icon on mobile toggle button

* Improve cluster search: searchable titles and scroll to match

- Make cluster titles and tab labels searchable (e.g., searching "QRP"
  now finds the "QRPs" tab)
- Search section description text outside of tabs
- Scroll to the first highlighted match within a tab pane instead of
  the cluster section top
- Handle section-level results (description/title matches)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Per-paragraph results, live search, fix cluster title display

- Show one result per matching paragraph/reference instead of per tab
- Filter out nested blocks (p inside li) to avoid duplicate results
- Fix cluster name: use h1 (actual title) not h3 ("Description")
- Live search as user types with 300ms debounce
- Make tab labels searchable (e.g. "QRP" finds "QRPs" tab)
- Scroll to the specific matched element, not the section top

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Extend cluster search to include titles and descriptions

The search previously only searched within tab panes. Now also searches
cluster titles and description content outside tabs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Richard Dushime <45734838+richarddushime@users.noreply.github.com>
Co-authored-by: richarddushime <mudaherarich@gmail.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants